Skip to content

opencode: init #1945

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 3 commits into from
Oct 21, 2025
Merged

opencode: init #1945

merged 3 commits into from
Oct 21, 2025

Conversation

@arunoruto
Copy link
Contributor

@arunoruto arunoruto commented Oct 18, 2025 •
edited
Loading

Adding opencode theming using stylix.
This is based on the example nord theme from the website.

@stylix-automation stylix-automation bot added topic: home-manager Home Manager target topic: testbed Testbed changes topic: flake /flake.nix, /flake.lock, and /flake/ subsystems topic: modules /modules/ subsystem labels Oct 18, 2025
trueNAHO
trueNAHO requested changes Oct 19, 2025
View reviewed changes
trueNAHO
trueNAHO reviewed Oct 19, 2025
View reviewed changes
@arunoruto arunoruto force-pushed the opencode branch 2 times, most recently from 86c64d7 to 31da77f Compare October 19, 2025 22:07
trueNAHO
trueNAHO approved these changes Oct 20, 2025
View reviewed changes
Copy link
Member

@trueNAHO trueNAHO left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@arunoruto, should we do anything about the unresolved discussions, or is this ready to merge?

@arunoruto
Copy link
Contributor Author

arunoruto commented Oct 20, 2025 •
edited
Loading

@arunoruto, should we do anything about the unresolved discussions, or is this ready to merge?

You mean regarding the vivid PR? I will change the commits and remove the maintainers change there.

@trueNAHO
Copy link
Member

trueNAHO commented Oct 20, 2025

[arunoruto], should we do anything about the unresolved discussions, or is this ready to merge?

You mean regarding the vivid PR? I will change the commits and remove the maintainers change there.

I was referring to #1945 (comment) and #1945 (comment). With #1945 (comment) resolved, I mean if my testing from #1945 (comment) is correct.

@arunoruto
Copy link
Contributor Author

arunoruto commented Oct 20, 2025

I was referring to #1945 (comment) and #1945 (comment). With #1945 (comment) resolved, I mean if my testing from #1945 (comment) is correct.

I just tested the change on my local system and it looks good to me!

@trueNAHO trueNAHO merged commit b81dc0a into nix-community:master Oct 21, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@trueNAHO trueNAHO trueNAHO approved these changes

Assignees

No one assigned

Labels

topic: flake /flake.nix, /flake.lock, and /flake/ subsystems topic: home-manager Home Manager target topic: modules /modules/ subsystem topic: testbed Testbed changes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@arunoruto @trueNAHO